cssstyleproperty: Remove base argument

from parse functions.

cssstylefuncs: Remove base argument

The GtkCssParser keeps track of the base file now.

cssshorthand: Don't pass base file anymore

It's part of the parser now.

cssimage: No need to pass base file anymore

The file is now part of the GtkCssParser

cssparser: Change APIs that work with files

Now that we keep track of the parser's file, we don't require it to be
passed anymore.

cssparser: Keep track of the file

Revert "cssprovider: Remove unused members from scanner"

This reverts commit b501e64a8a78e78c49b0abd1f34fc554ccacfd65.

cssprovider: Remove unused members from scanner

Now that the parser takes care of the file, we don't need to keep track
of it ourselves anymore.

[broadway] Fixup build

Seems the return value of device_query_state was removed, it was
TRUE always anyway.

gtk: Fix warnings for some uses of GtkLinkButton

Not setting a URI but catching the activate-link signal is a
valid use of GtkLinkButton, but we shouldn't allow showing a
popup menu which offers to copy the URI if there's none.

quartz: add a special case to GtkIMContextSimply for entering '"'

Turn dead_doubleacute plus space into '"' and not into a double
acute because that's the way to enter double quotes on the
US-International keyboard layout.
(cherry picked from commit 71164e57b9b999f07a613806058ee87b9cbf882d)

iconview: Don't draw unallocated items

This is more a workaround for Clutter-GTK than anything else. Most of
all, it makes gnome-boxes start again.

l10n: updated Italian translation

quartz: handle yet another dead tilde variant

which is delivered by the US-International keyboard layout.

gtk: Add private headers to EXTRA_DIST

This got lost with 985881ffcc315e20012a42dfba6cf5ab0046a92c

iconview: Only compute item sizes in one place

Code was using different places to compute the icon size when layouting
and when doing size requests. This resulted in non-matching behavior in
obscure cases. And that lead to iconviews that were too small.

iconview: Bring back per-row contexts

Apparently GtkCellAreaContext is broken enough to not allow multiple
allocations...

iconview: Move wrap width adjustment to size request code

stylecontext: Make queue_invalidate() always work

Handle both the case where a widget is set and also the case where a
widget path is set.

immulticontext: Always check if the global context id changed

This makes sure that if the gtk-im-module setting changes we update
our internal state immediately on the next event whichever it is.

In particular this fixes the case of the gtk-im-module setting
changing while the user is typing and the slave context remaining the
same, effectively ignoring the setting change.

https://bugzilla.gnome.org/show_bug.cgi?id=675365

infobar: set default border/spacing on action/content areas

Commit cdf473ec100d23ea6a9a8750ad301307a116b674 deprecated the style
properties associated with the container border width and box spacing of
the action and content areas, in favour of using
gtk_container_set_border_width() and gtk_box_set_spacing() on the
widgets themselves, but failed to initialize those values to the
defaults.

[Bug 675501] gtkquartz.h is not in the gtk+-3.5.2.tar.xz archive

Rearrange the gtk_private_h_sources so that they're all added to EXTRA_DIST instead of only the ones included in the current build.

Rename gtkquartz-menu to gtkmodelmenu-quartz

Makes name consistent with other quartz-only modules and makes it clear that this works with the GMenuModel system rather than the older GtkMenu system.

bgo#675421 - Set the wm_role on GtkFileChooserDialog

The window's role is 'GtkFileChooserDialog', so that window managers can match it
for positioning.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

sizerequest: Improve warning message

Actually print out the function we're warning about

iconview: Fix negative natural size request on empty iconview

broadway: Fix vfunc prototype

build: Don't invoke pkg-config directly

Instead, always use PKG_CHECK_MODULES(). That macro actually gets it
right. In particular the erroring out part when you miss xkbcommon or
wayland-client.

iconview: Don't expand items to more than natural size

This ensures that items stay left-aligned instead of slowly expanding into
empty space when widening the iconview. It's also what the iconview did
pre-refactoring.

Note that for cases where natural width != minimum width, the cells
might still expand and shrink back.

tests: Remove unused include

Fixes build

iconview: Redo layouting

iconview: Respect item width in size requestr code

iconview: Reduce complexity of sizing code

Always assume max-columns and min-rows. The old approach was kinda
insane.

As an example, try to write an algorithm that optimizes the minimum size
for infinite (take a reasonably large number like 2520) word-wrapped
Monospace text cells containing the text "XXXXX XXX XXX XXXXX" (keep in
mind that this is the easy problem, because it's assuming equal cell
renderers). There's 4 ways to reasonably lay out this text:
19 glyphs (19x1):
  XXXXX XXX XXX XXXXX
18 glyphs (9x2):
  XXXXX XXX
  XXX XXXXX
21 glyphs (7x3):
  XXXXX
  XXX XXX
  XXXXX
20 glyphs (5x4):
  XXXXX
  XXX
  XXX
  XXXXX
The best thing to do usually is using the 9x2 approach, but that's
neither the one using the natural nor the one using the minimum size.

As a side note, this does not include spacing and padding, which might
also influence the decision. Nor does it include height-for-width
considerations. Look at this table (numbers given in glyphs, not pixels,
as for pixel-sizes it gets even more interesting):
  given  best solution
  width  columns  sizing  glyphs per cell
   6      1       6x4           20
   7      1       7x3           21
   8      1       7x3           24
   9      1       9x2           18
  10      1/2     9x2/5x4       20
  11      1/2     9x2/5x4       22
  12      1/2     9x2/5x4       24
  13      1/2     9x2/5x4       26
  14      2       7x3           21
  15      3       5x4           20
  16      3       5x4           21.3
  17      3       5x4           22.7
  18      2       9x2           18
  19      1/2    19x1/8x2       19
  20      1/2/4  19x1/8x2/5x4   20
  21      1-4     any           21
  22      1-4     any           22
  23      1-4     any           23
  24      1-4     any           24
  25      5       5x4           20
  26      5       5x4           20.8
  27      3       9x2           18
  28      3       9x2           18.7
  29      3       9x2           19.3
  30      3/6     9x2/5x4       20

Now of course, nobody wants the number of columns to randomly change in
inexplicable ways while they enlarge or shrink an iconview, so we not
only have to optimize for smallest or other size measurements, but we
also have to optimize for "most pleasing to the eye".

And last but not least, I'd like to once again remind you - if you kept
up until now - that this discussion was for identically-sized cells
only.

tests: Make testcellarea use an iconview

iconview: Redo size requests

Instead of just returning the last allocated numbers, we now compute the
proper sizes from scratch. This is a bit less trivial, but it results in
proper height-for-width handling.

iconview: Don't cache iters

This is a huge quest to remove all caching from GtkIconview to simplify
the code. As it turns out, iconview performance is a joke, so the caches
are kinda unnecessary.

If we need caching, we can add it in a useful way later.

iconview: Remove needless requirement

If we use
  &item->cell_area
instead of
  (GdkRectangle *) item
there is no need anymore to keep the cell_area as the first member of
the the item. And we cget compile-time checks for changes to the item
struct.

gtk_entry_[get|set]_inner_border(): Clarify the deprecation docs.

[socket]: fix gtk-doc warnings.

gdk: do not send leave event on touchpad button release

This is along the changes introduced in
bd55519f7e6bb5da2fdf6f5e99bd79213059f81b,
A TOUCHPAD device behaves like a mouse"

https://bugzilla.gnome.org/show_bug.cgi?id=674549

Updated Hebrew translation.

Updated Galician translations

themingengine: don't hardcode the slider lines in the theming engine

It's wrong to hardcode the slider lines here; those should be rendered
as an additional background layer using background-image if the theme
specifies so.

https://bugzilla.gnome.org/show_bug.cgi?id=652304

entry-completion: set size request on the toplevel window

Instead of setting it on the child scrolled window. This is needed
because the whole window's allocation must be equal to the one of the
entry (in case the popup-set-width property is TRUE); if we set the size
request on a children of the window, there might be other children with
borders/paddings in between the toplevel and the child we set the size
request too, which will break alignment.

https://bugzilla.gnome.org/show_bug.cgi?id=672947

entry-completion: don't subtract entry borders from popup allocation

The completion popup should be aligned with the GtkEntry allocation, not
less than that.

https://bugzilla.gnome.org/show_bug.cgi?id=672947

Updated Spanish translation

cellrenderertext: Add placeholder-text property

It works similar to placeholder text in a GtkEntry, if the cell renderer
is editable and it's empty, the placeholder text is displayed.
https://bugzilla.gnome.org/show_bug.cgi?id=645514

docs: Improve GtkMessageDialog description

Add mention of default setting of GtkWindow:skip-taskbar-hint.

https://bugzilla.gnome.org/show_bug.cgi?id=587873

docs: Move GtkSelectionData to gtk3-Selections

Put the (albeit empty) documentation for the GtkSelectionData struct
with the gtk_selection_data_*() methods.

https://bugzilla.gnome.org/show_bug.cgi?id=663072

docs: Correct gtk_tree_row_reference_new() description

Remove the text that mentions listening to tree model signals, and
instead say that any changes are automatically propagated.

https://bugzilla.gnome.org/show_bug.cgi?id=478344

Add support for disabling middle click paste

Adds a gtk-enable-primary-paste setting that is backed by
and X setting, and make GtkEntry and GtkTextView respect it.

https://bugzilla.gnome.org/show_bug.cgi?id=665243

printing: Rename GTK_UNIT_PIXEL to GTK_UNIT_NONE

...and document that many functions don't accept "none" as a unit.

https://bugzilla.gnome.org/show_bug.cgi?id=460908

gdk: add bindings-friendly alias for GDK_EVENT_2BUTTON_PRESS

GDK_EVENT_2BUTTON_PRESS and GDK_EVENT_3BUTTON_PRESS can't be used from
some bindings because they'd translate to something syntactically
invalid. Add GDK_EVENT_DOUBLE_BUTTON_PRESS and
GDK_EVENT_TRIPLE_BUTTON_PRESS aliases to work around that.

https://bugzilla.gnome.org/show_bug.cgi?id=671025

window: Remove needless call to ensure_state_flag_backdrop() on map

This call was forcing needless work since gtk_window_map() already
does a gdk_window_show() which initially sets GDK_WINDOW_STATE_FOCUSED
that we then handle regularly on the widget's window state event
handler.

https://bugzilla.gnome.org/show_bug.cgi?id=673237

build: Fix out-of-tree build of widget-factory demo application

https://bugzilla.gnome.org/show_bug.cgi?id=674192

Updated Dutch po-properties translation by Rachid, reviewed by Reinout
van Schouwen.

Fix unclosed comments

colorbutton: don't hardcode a 16px margin around the color sample

Since themes might want different paddings around the color sample
according to where it's being used, don't hardcode a 16px one here.
The theme can specify a padding to get the same effect.

Reported by Christian Persch <chpe@gnome.org>

application: Fix build without x11

application: Add unique IDs for GtkApplicationWindow

This will allow to refer to specific GtkApplicationWindows remotely by ID.

https://bugzilla.gnome.org/show_bug.cgi?id=674409

fontsel: Add a deprecation note in the constructor docs

To match the compiler attribute, otherwise the introspection scanner
will not pick up the deprecation.

Bug 670499-gtk/fallback-c89.c: Add fallback for nearbyint()

This adds a C89 implementation for nearbyint() as it is a function that
is only made available in C99.

Bug 670499-gdk/gdkrgba.c: Include fallback-c89.c

Include (gdk/)fallback-c89.c as isnan() and isinf() is used.

Bug 670499-Add gdk/fallback-c89.c

Add a fallback-c89.c for the gdk/ subdirectory as there is code that uses
functions that are introduced by C99.  This currently adds fallback
implementations for MSVC for isnan() and isinf()

Dist this "new" source file as well

Bug 670499-configure.ac: Check for nearbyint(), isinf() and isnan()

nearbyint(), isinf() and isnan() are C99 functions, so check for them.

Also clean up configure.ac a little bit as the checks for rint() and
round() can be a bit simpler, according to Matthias' suggestions.

Bug 670449-gtk/gtksettings.c: Fix build on Windows

On Windows, gtkwin32themeprivate.h is needed as
_gtk_win32_theme_get_default() is called on that platform to avoid C4013
warnings/errors (aka implicit declaration of ... for GCC folks).

pathbar: fix a copy/paste error from last commit

We need to update the child pointer to the current iterator when going
trough the button list.

stylecontext: Disallow adding/removing GtkSettings style providers

Adding "system" providers like the GtkSettings object shouldn't be
allowed at all, so ensure that it indeed is not.

pathbar: Only invalidate styles in allocate if stuff really changed

I still don't like queueing restyles in size_allocate(), but I can't
come up with a better mechanims for the way the pathbar works today.

Updated Greek translation

debug: Add GTK_DEBUG=no-css-cache

See inline comments for what it does. Its main use is figuring out if
something has been caused by GTK's caching of CSS properties or if it's
a different problem.

build: Set G_ENABLE_DEBUG for minimal --enable-debug=minimum

With that flag set, we get GDK_DEBUG and GTK_DEBUG support. And those
provide useful features we want when debugging.

window: Validate style context before first allocation

When we don't do that, we completely botch sizing popups. Not good.

Fixes remaining failing reftests

reftests: Remove unneeded event boxes

The event boxes break when backgrounds are transparent.

Fixes border-image-repeat.ui reftest.

reftests: Make reset css include new CSS properties

cssshadow: plug a memory leak

Bump version

Bump to 3.5.2

Add some more documentation about GDK_SCROLL_SMOOTH

cssshadow: Default fallback color is transparent

... not the current color. Fixes unresolvable.ui test.

cssmatcher: Handle case of empty widget path

This is tested by the stylecontext test, but doesn't appear in practice.

cssprovider: Actually match the state for style properties

Copy/paste error when refactoring.

Caught by stylecontext test.

versionmacros: min-require the current stable version

.. instead of the previous stable version.

This ensures that if we use GDK_DEPRECATED_IN_3_6, it will actually emit
a warning in GTK 3.5, and not wait with that until GTK 3.7. This is
particularly useful for stuff that gets deprecated right now. This code
should emit warnings right now, so we know what we're doing while
deprecating.

docs: Fix copy/paste typo

gtk: Get rid of deprecation warnings

tests: Ifdef out unused code

The test using those functions was disabled in
b310ba29d0c14cdd566b4af1dd10599735980985

build: Remove now-unused GTK_DISABLE_DEPRECATED undefs

Update reference output for css parser tests

Seems these were not kept up-to-date with the recent
CSS parser changes.

GtkApplication: adjust to GApplication changes

GApplication now makes the session bus and object path available as a
public API on the application instance.  Use that instead of trying to
guess values for ourselves.

This causes this version of Gtk+ to depend on GLib 2.32.2, so bumping
version dependency accordingly.

https://bugzilla.gnome.org/show_bug.cgi?id=671249

GtkApplication: fixes for NULL session bus

We currently have a couple of cases where GtkApplication assumes that
the session bus will be non-NULL causing critical error output or (in
the case of trying to publish menus) an infinite loop.

Three fixes:

 - if the session bus is NULL due to not having registered the
   GtkApplication yet then give a g_critical on the entry point to the
   menu setters instead of going into an infinite loop.  Document this.

 - check for NULL session bus even when calling the menu setters at the
   right time in order to prevent the infinite loop for
   non-programer-error cases (ie: because we had trouble connecting to
   the session bus)

 - check for NULL session bus when publishing the X11 properties on the
   GtkApplicationWindow and skip publishing them if we're not on the bus

https://bugzilla.gnome.org/show_bug.cgi?id=671249

infobar: deprecate broken GtkInfoBar style properties

The way we use these style properties to set regular properties on
containers accessible from the public API is really just broken, and
could lead to undefined values for the spacing and border-width
container properties (since they could be set from public API and then
changed from under in a style_update handler from GTK).

Take this as an occasion to deprecate these style properties, which do
not make a lot of sense anyway, now that GtkInfoBar supports regular CSS
padding and border.

GtkPlug: fix handling of key events for different layouts

GtkPlug directly handles X KeyPress/Release events, instead of using
translation in GDK (which expects XI2 events for XI2). When this
was done, the handling of the group was stubbed out and never replaced.

Export gdk_keymap_x11_group_for_state() and gdk_keymap_x11_is_modifier()
so we can fill out the fields correctly.

https://bugzilla.gnome.org/show_bug.cgi?id=675167

Fix info-bar reftest

InfoBar must take into account the border and padding when requesting
its allocation, since it is then drawing them. Besides, the border and
background should always be drawn, even when the message type is OTHER.

Fix infobar reftest for label background changes

Fix the reftest after previous commit

https://bugzilla.gnome.org/show_bug.cgi?id=670743

reftests: fix background-size reftest

We have to be careful and not set the background on the labels
themselves.

https://bugzilla.gnome.org/show_bug.cgi?id=670743

Paint label background

Make GtkLabel respect background css properties.

https://bugzilla.gnome.org/show_bug.cgi?id=670743

themingbackground: fix copy/paste typo

We need to fetch the area value for the origin, not for the clip.

Disable configuration-dependent keymap tests for now

These need to be made independent of the xkb configuration somehow.
As things are now, they will either fail when run on a naked X
server in make check, or fail when run in my session.

Fix build with --enable-debug

Updates for 3.5.1